-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[JOH-26] Sign out #5
Conversation
JOH-26 Auth and User endpoints
Signup - POST /auth/signup Signin - POST /auth/signin Signout - POST /auth/signout Validate - POST /auth/verify RefreshToken - POST /auth/refreshToken |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
src/app/service/auth/auth.service.go
Outdated
if err != nil { | ||
st, _ := status.FromError(err) | ||
switch st.Code() { | ||
case codes.Unavailable: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we put a log here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, I think so.
Demo
Firstly, we need to sign in to get access token, then we put the token in the headers of request
Then auth session in database would be deleted.